-
Notifications
You must be signed in to change notification settings - Fork 32
Public inits, mutable properties and full Codable support #89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Public inits, mutable properties and full Codable support #89
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor nits
I'm still a little reluctant to open these types up but understand the desire and will likely out voted on this 😅
Sources/WebAuthn/Helpers/KeyedDecodingContainer+decodeURLEncoded.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one nit and a comment that's probably fine about a public type
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Helpers/KeyedDecodingContainer+decodeURLEncoded.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly formatting changes left and a few consistency issues, but otherwise looks good to me!
Sources/WebAuthn/Ceremonies/Authentication/AuthenticatorAssertionResponse.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Authentication/PublicKeyCredentialRequestOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Authentication/PublicKeyCredentialRequestOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Authentication/PublicKeyCredentialRequestOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Registration/PublicKeyCredentialCreationOptions.swift
Outdated
Show resolved
Hide resolved
Sources/WebAuthn/Ceremonies/Authentication/AuthenticationCredential.swift
Outdated
Show resolved
Hide resolved
Filed the failing 6.1/6.2 builds here: |
Public inits, mutable properties and full Codable support